Refactor split RT-DATA into separate modules/files #515
+706
−785
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
rt/value.md
contains theValue
sort andTypedLocal
with its subsortsrt/numbers.md
module contains code related to decoding and casting integers, and helper functionsrt/configuration.md
Most of this is just moving existing code around into new files and adjusting imports.
The
RT-DATA-LOW
andRT-DATA-HIGH
modules were removed, moving code fromHIGH
intoRT-DATA
(RT-DATA-LOW
was never actually used).New module import graph (omitting
domains.md
modules):We can split
rt/data.md
into more parts, e.g.Rvalue
evaluation (in general), which could also move toKMIR-CONTROL-FLOW
insidekmir.md
BinaryOp
,UnaryOp
(which are specificRvalue
s)